Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

just panda #24

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

just panda #24

wants to merge 6 commits into from

Conversation

var114
Copy link

@var114 var114 commented Oct 31, 2013

No description provided.

@jwo
Copy link
Member

jwo commented Nov 2, 2013

Hi! Just wanted to let you know I got the Pull Request... I looked it over and it looks good. I'll be doing a more intensive review in the next 1-2 days. until then: 👍

@@ -1,13 +0,0 @@
class CreateNetworks < ActiveRecord::Migration
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you delete this migration?

@jwo
Copy link
Member

jwo commented Nov 4, 2013

Ok so I left some questions --- overall a nice job! Yay!

@var114
Copy link
Author

var114 commented Nov 6, 2013

Done

@@ -1,50 +0,0 @@
Episode3: Watchman -- Datastores in YML and Postgres
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why delete README? if you're going to, you should replace it witch your own readme.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Jesse,

I’m not sure why the README page got axed.
I was just probably frustrated and took it out on the README page.

I put it back and added your suggestions.

I have a general question though about pull requests.
If I send you an initial request, and then alter my master…will it automatically change it in the pull request I sent you?

On Nov 6, 2013, at 10:42 AM, Jesse Wolgamott [email protected] wrote:

In README.md:

@@ -1,50 +0,0 @@
-Episode3: Watchman -- Datastores in YML and Postgres
why delete README? if you're going to, you should replace it witch your own readme.


Reply to this email directly or view it on GitHub.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep. as you push more code to github in your repo, if you have an existing pull-request that will get updated.

If you're feeling advanced, you can have multiple branches that each could have their own pull request (though that gets complicated easily). For more on git:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants